Size Report

Startup median (7 runs, lower is better):

Top changed chunks:

I do not see a routing blocker in the doctor command path: the command facet projects through the daemon registry, and the Android probes still run under the request-scoped provider path via the adb executor scope.

One thing I would address before merge: src/daemon/handlers/session-doctor.ts lands as a 571-line handler and mixes option parsing, session/device selection, app discovery, Android foreground/reverse/animation probes, RN overlay checks, Metro probing, output ordering, and progress emission. The repo agent guide asks us to extract focused submodules once implementation files grow past 500 LOC. Please split the probe families out before merge, for example keeping session-doctor.ts as the orchestrator and moving Android/Metro/RN checks into focused session-doctor-* modules or a small doctor-checks/ group. That will make follow-up doctor checks reviewable without growing the handler further.

Re-check after the latest push: this still needs the split before I would call it merge-ready. src/daemon/handlers/session-doctor.ts is now 599 LOC, and the Android/Metro/RN/session/device checks are still all in that handler. CI is green and the route/provider coverage looks fine, but the previous maintainability finding is unresolved.

GitHub now reports this branch as conflicted against main after #884 merged. Please rebase, resolve the conflicts, and reply with what changed plus any validation you reran. I am holding re-review until the branch is clean again.

Current head 51fb503 has failed required checks:

• Unit Tests: https://github.com/callstack/agent-device/actions/runs/28471789769/job/84386075617
• Integration Tests: https://github.com/callstack/agent-device/actions/runs/28471789769/job/84386075606
• Coverage: https://github.com/callstack/agent-device/actions/runs/28471789769/job/84386075609

Smoke checks are still settling, but these CI failures need attention before review/readiness. Please address or rerun if the failures are known infra flakes, then reply with what changed and what was validated.